Skip to content

Bump click minimum version to 8.3.1#1556

Open
stratakis wants to merge 1 commit intodbcli:mainfrom
stratakis:click
Open

Bump click minimum version to 8.3.1#1556
stratakis wants to merge 1 commit intodbcli:mainfrom
stratakis:click

Conversation

@stratakis
Copy link

Click 8.1.8 through 8.3.0 have broken pager invocation for multi-argument PAGER values, which causes behave test failures.

Fixed on pallets/click@7db1f20

  • I've added my name to the AUTHORS file (or it's already there).

Click 8.1.8 through 8.3.0 have broken pager invocation for
multi-argument PAGER values, which causes behave test failures.

Fixed on pallets/click@7db1f20
dependencies = [
"pgspecial>=2.0.0",
"click >= 4.1,<8.1.8",
"click >= 8.3.1",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe try this?

Suggested change
"click >= 8.3.1",
"click >= 4.1,<8.3.2",

@meeuw
Copy link
Contributor

meeuw commented Feb 19, 2026

@stratakis thanks for your PR, I approved the CI tests.

For anyone reading this, rationale can be found here:

Required for click's rebase to 8.3.1.

https://src.fedoraproject.org/rpms/pgcli/pull-request/14

DiegoDAF added a commit to DiegoDAF/pgcli.daf that referenced this pull request Feb 19, 2026
click 8.1.8-8.2.x had broken pager invocation for multi-argument PAGER
values. click 8.3.1 includes the fix (pallets/click@7db1f20).
Removes the defensive <8.1.8 upper bound.

Ref: dbcli#1556

Made with ❤️ and 🤖 Claude
Copy link

@DiegoDAF DiegoDAF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed working. We've been running with click 8.3.1 on our fork with the full test suite passing (2759 tests).

The old <8.1.8 upper bound was a workaround for the broken pager in 8.1.8-8.2.x. This is the proper fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments